Enough with the beeping!

Don't beep when modifiers are released in entries.
This was an inadvertent change that snuck in with
the emoji support.

Also, don't beep while entering an emoji name.
There is entirely too much beeping here.

Use the right gettext domain

Most of our .ui files were still using the gtk30 domain,
despite the library having switched to gtk40. Fix this.

AccelLabel: Fix displaying accel unichars >= 0x80

In GTK+ 2, the ch < 0x80 was ORd with klass->latin1_to_char, and that
was unconditionally set to TRUE in the class init function, so
effectively the ch < 0x80 never mattered before or served any purpose.

When klass->latin1_to_char was deleted from the class in commit
f760538f17673c5bd7fec792be2f1abf8148fc32, this check’s sense changed.

The resuls was that accel keyvals with gunichar value >= 0x80 stopped
being rendered as symbols, instead falling back to their keysym name.
Instead of recognisable symbols for these, we get raw, often obscure,
and untranslatable keysym names. This breaks accessibility as well as
client users who may be parsing such accels and migrating from GTK+ 2.

So, remove the < 0x80 to restore the behaviour from before said commit.

https://bugzilla.gnome.org/show_bug.cgi?id=783906

CenterBox: minor tweaks to doc blurb

CenterBox: Add missing nullable annotations

https://bugzilla.gnome.org/show_bug.cgi?id=785280

CenterBox: Guard against directly including header

https://bugzilla.gnome.org/show_bug.cgi?id=785280

centerbox: Add G_BEGIN_DECLS and G_END_DECLS

https://bugzilla.gnome.org/show_bug.cgi?id=785280

Add support for entering emoji by name

This commit adds some basic support for entering emoji by name
to GtkIMContextSimple. To begin an emoji sequence, use Ctrl-Shift-e
instead of Ctrl-Shift-u that is used for hex input. Otherwise, the
behavior is the same: you can can let go of the modifier keys and
end the sequence with space or enter, or hold on to the modifier
keys and end the sequence by releasing them.

Only a limited, fixed set of names is supported at this time, see
the GtkIMContextSimple docs for a full list.

gskrendernodeimpl: Fix leak of cairo_pattern_t

It was not destroyed after setting it on the cairo_context_t, as it is
in every other case.

https://bugzilla.gnome.org/show_bug.cgi?id=785793

Window: Also quote Window address in Popover error

This is symmetrical with how we already do that for the Popover, and it
could be useful for debugging.

widget-factory: Add tooltip to Entry icon

just to show we can.

Fix some nullable Returns annotations

based on reports and patches by Iñaki García Etxebarria:

https://bugzilla.gnome.org/show_bug.cgi?id=781246
https://bugzilla.gnome.org/show_bug.cgi?id=785423

inspector: Fix object title label alignment

Re-center it after the center-child removal in GtkBox and the addition
of GtkCenterBox.

GtkFixed: Update documentation intro

• Add GtkLayout as a @See_also since it includes fixed-pos functionality
• Drop mention of the long-gone Linux framebuffer port
• Explain how to work around the problems with RTL text

Updated Czech translation

Layout: Tweak documentation blurb

Being addable to a ScrolledWindow is not interesting; now that SW
auto-adds a Viewport if needed, so can DrawingArea and any other widget.

Mention GtkFixed in case the reader just wants that bit of functionality

entry: get_icon_area returns in entry coordinates

This fixes the popover positions in the gtk4-demo/popovers, and it also
just makes sense. Also document what the returned coordinates are
relatiev to.

popover: Fix arrow position

Remove all the manual css margin handling since that's being done for us
automatically now.

Update POTFILES.in

eventbox: Remove

texthandle: Don't use GtkEventBox

testsuite: Don't use GtkEventBox in gestures test

a11y: Remove check for GtkEventBox

docs: Don't use GtkEventBox

testsuite: Remove GtkEventBox from a11y test

iconbrowser: Remove GtkEventBox usage

gtk-demo: Get rid of GtkEventBox

cellaccel: Get rid of GtkEventBox

Can't test the code atm because treeview editing is broken, let's hope
it works once that's fixed.

placessidebar: Remove EventBox usage

placesview: Remove event box usage

meson: Add missing Wayland protocol

GDK depends on a new Wayland protocol.

wayland: add shortcut inhibitor support

This adds support for the shortcut inhibitor protocol in gdk/wayland
backend.

A shortcut inhibitor request is issued from the gdk wayland backend for
both the older, deprecated API gdk_device_grab() and the new gdk seat
API gdk_seat_grab(), but only if the requested capability is for the
keyboard only.

https://bugzilla.gnome.org/show_bug.cgi?id=783343

reftests: Always wait for rendering to settle

We wait for a few 100ms for rendering to settle in various WMs. So far
we only did that for windows that were controlled by the WM (aka
toplevels).

With modern compositing gnome-shell however, this now also applies to
override-redirect windows, so we now wait there, too.

This makes the reftests a lot slower, but they now actually work when
running make check in gnome-shell.

reftests: Remove event box usage

testuite: Remove EventBox from builder test

Updated Slovenian translation

widget: Constify state parameter of propagate_state

Makes sense.

widget: Don't try to propagate 0 state flags

This happens quite often and has no effect anyway.

widget: Fix size_allocate's out_clip annotation

testgtk: Actually expand the scrolled window

testgtk: Get rid of GtkEventBox

tests: Get rid of GtkEventBox

bindings: Avoid iterating array if it is never set

If query.return_type is not one we want, binding_compose_params() is
not called, and so params remains a NULL pointer. However, the code was
then unconditionally iterating it regardless. Don't if it is still NULL.

CID 1452218 (#1 of 1): Explicit null dereferenced (FORWARD_NULL)
15. var_deref_op: Dereferencing null pointer params.

cssselector: Ensure we do not index out of bounds

This would only happen if the last element was deprecated, but it should
be avoided anyway.

CID 1388852 (#1 of 1): Out-of-bounds read (OVERRUN)
12. overrun-local: Overrunning array pseudo_classes of 16 32-byte
elements at element index 16 (byte offset 512) using index i + 1U (which
evaluates to 16).

Notebook: Remove null check after dereference

This function clearly assumes the parameter children cannot be NULL, and
the call sites seem to perform enough checks to confirm this.

CID 1388869 (#1 of 1): Dereference before null check (REVERSE_INULL)
check_after_deref: Null-checking children suggests that it may be null,
but it has already been dereferenced on all paths leading to the check.

Calendar: Add default case to appease Coverity

CID 1432024 (#1 of 1): Uninitialized scalar variable (UNINIT)
2. uninit_use_in_call: Using uninitialized value rect.x when calling
calendar_arrow_rectangle.

Add a default case to the switch which will bail out with
g_assert_not_reached(), which should reassure Coverity that the method
is always called with a valid value that is handled in the switch.

cssbordervalue: Fix possible use of uninitialised

If value->values[i] is NULL, then values[i] was left uninitialised.
The code then reads each element of values[].

CID 1432029 (#1 of 1): Uninitialized pointer read (UNINIT)
11. uninit_use: Using uninitialized value values[i].

Stack: Fix copy-paste error

This fixes Coverity CID 1452217.

Entry: Fix unset icon tooltip hiding Entry tooltip

Our ::query-tooltip handler first checks whether the pointer is over any
of the icons, returning their tooltip if so, and if not chains up to
Widget::query-tooltip in order to show the text for the widget overall.

But ensure_has_tooltip(), which exists to update :has-tooltip based on
whether ::query-tooltip is needed, only set :has-tooltip to TRUE if any
icon had a tooltip, without caring whether the widget as a whole does.

That is asymmetrical and meant that if the Entry had a tooltip, but
subsequently all icons had their tooltips unset, :has-tooltip would be
set to FALSE, and hence the tooltip for the widget would become lost.

The fix is to set :has-tooltip to TRUE if the widget has a tooltip of
its own, and we only need to check the icons if that is not the case.

https://bugzilla.gnome.org/show_bug.cgi?id=785672

Entry: Warn about corner case hiding icon tooltips

https://bugzilla.gnome.org/show_bug.cgi?id=785672#c4

widget: Unset PRELIGHT and ACTIVE state in _set_sensitive

They both depend on input, which an insensitive widget does not receive.

button: Stop fiddling with PRELIGHT state

This is done automatically now and it's wrong here since it gets
executed after gesture release.

scale: Allocate the value next to the slider

... when vertical. Also fixes usage of the uninitialized value_alloc.y.

Visual Studio builds: Adapt to new glib-mkenums

glib-mkenums is now done in Python, but since the Visual Studio build
environment (cmd.exe) does not support shebang lines, we need to call
the interpretor explicitly to run the script.

This means that we need to update on how we generate
gsk/gskenumtypes.[c|h] in our projects, as at this point GTK+-3.91.x
does not require a GLib installation that ships with the Python-fied
glib-mkenums.  As a result, we adapt to this by first using Python
to call glib-mkenums.  If this fails (where the output file becomes 0
in size), then we use PERL to call the glib-mkenums script.  Note that
during the build this will cause a warning message to be displayed,
stating that '&' cannot be found, but due to the way Windows .bat script
are done, we need to live with that until a solution can be found on
this.

This is likely a problem that does not exist in the Meson builds, as
Meson will take care of calling the interpretor for us by looking at
the shebang lines for our case.

Also, clean up the .batin Windows batch script that is used to call
glib-mkenums by using a for loop in there.

testentryicons: Set tooltips on all icons

Just to test tooltips in all cases; what was already here
should have been sufficient, but this doesn't hurt.

While here, also add some instructive placeholder text.

https://bugzilla.gnome.org/show_bug.cgi?id=780938

MSVC builds: Allow different Python versions per toolset

Use conditionals to select the Python installation, so that we can stick
closer to the default Visual Studio versions used to compile each official
Python releases.

This means by default:
-2013 builds use Python 3.4.x, which is built with 2010
-2015 and 2017 builds use Python 3.6.x, which is built with 2015.

Also rename PythonPath/PythonPathX64 in the property sheets to
PythonDir/PythonDirX64 repsectively, as PythonPath is the envvar name
where additional Python modules is searched for, so we don't want to get
confused with it.

Last but not least, distinguish between the Python interpretors that are
used on x64 and x86/32-bit builds for generating the libgtk4.manifest
file and the gdbus-generated sources, for consistency reasons.

entry: Remove unused/duplicate widget in IconInfo

widget: Don't access highlight_resize in non-debug builds

The GtkWidgetPrivate struct only contains that member if G_ENABLE_DEBUG
is defined.

eventbox: Remove visible-window property

The GdkWindow that was supposed to back it is gone.

Updated Slovenian translation

Updated Slovenian translation

Updated Slovenian translation

gtklabel: Fix touch link handling under wayland

Refactor the code updating the active link under the current coordinates
into a separate function, and call it on GtkGestureMultiPress::pressed
so the link is updated on GDK_TOUCH_BEGIN. Based on a patch by
Jan-Michael Brummer <jan.brummer@tabos.org>.

https://bugzilla.gnome.org/show_bug.cgi?id=776903

gtklinkbutton: Set cursor on button's event window

It is not necessary to (re)set the cursor on every crossing
event, and can probably yield the wrong results if there are
multiple master devices involved. Just set it on init(), and
let the inner machinery update the cursor whenever necessary.

This patch is an adaption of commit 0daf79676 in gtk-3-22, the
side effects are not as bad here because the cursor was already
being set on the widget specifically instead of the parent
widget's, but there's still some nonetheless (plus, it's simpler)

https://bugzilla.gnome.org/show_bug.cgi?id=785375

wayland: Clear tablet tool cursor on proximity out

This ensures that the tablet tool will get the cursor updated even if it
happens to fall within the same window again on the next proximity in.

https://bugzilla.gnome.org/show_bug.cgi?id=785375

gdkwindow: Update cursor for tablet tools in the window

This check must be done explicitly on Wayland as the master device for
tablet tools differ from the Core Pointer. This ensures that whenever a
tablet tool is inside a window and the cursor is programmatically changed,
it will be visually updated too.

https://bugzilla.gnome.org/show_bug.cgi?id=785375

wayland: Observe GDK_SEAT_CAPABILITY_TABLET_STYLUS on gdk_seat_get_slaves()

This flag wasn't being honored so far...

Updated Spanish translation

Update Croatian translation

actionbar: Call g_object_class_install_properties()

https://bugzilla.gnome.org/show_bug.cgi?id=785318

Avoid the use of g_autofree

We can't use this in code that may be built on win32,
so lets just not use it at all.

widget: Stop invalidating recursively

updated Punjabi Translation

Update Friulian translation

notebook: Set tabs child visible before allocating a size

Otherwise we might skip them here, then set child-visible to TRUE and
then later snapshot them even though they have no valid size.

cssstylechange: Fix change printing

Only print the old value once, above the new one.

Update Indonesian translation

flowbox: Fix get_child_at_pos

testflowbox: hexpand the flowbox

tests: Include system headers appropriately

Don't include unistd.h on Windows, and include the appropriate headers on
Windows as necessary.

https://bugzilla.gnome.org/show_bug.cgi?id=773299

tests/rendernode-create-tests.c: Don't use VLAs

Replace uses of VLAs (variable-length arrays) using g_newa(), since
Visual Studio builds will unlikely ever support VLAs (which became optional
in C11).

https://bugzilla.gnome.org/show_bug.cgi?id=773299

gtk/gtkprintoperation-win32.c: Don't call gtk_widget_set_allocation()

This function was removed recently.

https://bugzilla.gnome.org/show_bug.cgi?id=773299

wayland: Implement support for tablet wheel scrolling

Adds support for creating scroll events from Wayland tablet wheel events.
Even though no Wacom tablet puck has a smooth-scrolling wheel, both event
types need to be generated to make the upper layers happy.

https://bugzilla.gnome.org/show_bug.cgi?id=783716

wayland: Make function to create scroll event more generic

Add GdkWaylandPointerData and GdkDevice arguments so it can be
used across master devices.

https://bugzilla.gnome.org/show_bug.cgi?id=783716

wayland: Get implicit grab serial information from tablet devices

If a tablet device is used to perform actions like window moving or resizing,
GTK must provide the correct implicit grab serial number over Wayland to Mutter
in order for the action to succeed. This commit adds tablet support to the
implicit serial getters.

https://bugzilla.gnome.org/show_bug.cgi?id=777333

gdkgears: Add window content spacing back

This was lost when border-width was removed.

overlay: Fix child widget allocation

Just adding the overlay's own position is wrong now.

overlay: Keep the main child at the bottom

Since gtk_bin_add does a gtk_widget_set_parent call, we cannot use it in
a GtkBin implementation that has multiple child widgets and cares about
their order.

Move GdkWindowWindowClass to private headers

This enum is no longer used in public api.

Drop an unused typedef

GdkWindowRedirect is not used anywhere anymore.

wayland: Throttle system bell requests

If a bad behaving application tries to make the window/display beep too
often, throttle the beep requests so that we don't end up filling the
Wayland socket queue.

The throttle is set to 50 beeps per second, which far more beeps than
will ever make any sense from a user experience point of view, but will
avoid terminating due to an excessive amount of requests.

https://bugzilla.gnome.org/show_bug.cgi?id=778188

wayland: Make beep requests go through the GdkDisplay

This way we can add things like throttling.

https://bugzilla.gnome.org/show_bug.cgi?id=778188

Summarize changes so far

This is a high-level view of what was just merged.

testgtk: Remove unused local variable

GtkStyleContext: Create path node in _new

So we can avoid creating a GtkCssPathNode in _init and then throwing it
away right after when using the _new_with_node constructor, which is the
one we use for all widgets.

GtkStyleContext: Remove property cache

It was only used for widget style properties which don't exist anymore.

widget-factory: fix page 3 widget expand

The treeview/listbox on the left side should not expand.

GdkWindow: Default to unfocused

Don't set the have_focused field of the window's toplevel to TRUE by
default and don't set the FOCUSED state in gdk_window_map. This a means
toplevel window's state is what the WM expects, and the FOCUSED state
will be set anyway when we map the window and receive a _NET_WM_STATE
message.

statusbar: Inherit from GtkWidget

modelmenuitem: Care about GtkAccelLabel in get_text